[ASAN] bootconfig: Drop a pointless strdup in parser
authorColin Walters <walters@verbum.org>
Tue, 29 Nov 2016 03:02:42 +0000 (22:02 -0500)
committerAtomic Bot <atomic-devel@projectatomic.io>
Wed, 30 Nov 2016 18:51:26 +0000 (18:51 +0000)
Not entirely sure how this was leaking, but anyways it showed
up in ASAN, and it's pointless to strdup here.

Closes: #598
Approved by: jlebon

src/libostree/ostree-bootconfig-parser.c

index f7728e42060cb7bd9f7771129cf8b2dd678fa463..a2ac1072fd7c780b13ef9cdaaa487734ad368c1a 100644 (file)
@@ -28,7 +28,7 @@ struct _OstreeBootconfigParser
   GObject       parent_instance;
 
   gboolean      parsed;
-  char         *separators;
+  const char   *separators;
 
   GHashTable   *options;
   GPtrArray    *lines;
@@ -235,7 +235,6 @@ ostree_bootconfig_parser_finalize (GObject *object)
 
   g_hash_table_unref (self->options);
   g_ptr_array_unref (self->lines);
-  g_free (self->separators);
 
   G_OBJECT_CLASS (ostree_bootconfig_parser_parent_class)->finalize (object);
 }
@@ -261,6 +260,6 @@ ostree_bootconfig_parser_new (void)
   OstreeBootconfigParser *self = NULL;
 
   self = g_object_new (OSTREE_TYPE_BOOTCONFIG_PARSER, NULL);
-  self->separators = g_strdup (" \t");
+  self->separators = " \t";
   return self;
 }